[perf] test MCP510 #113382

lqd · 2023-07-05T20:17:41Z

lqd · 2023-07-05T20:23:07Z

Let's see if the bootstrapping cfgs are correctly set up.

lqd · 2023-07-05T23:01:59Z

Let's see if the bootstrapping cfgs are correctly set up.

$  cargo +4cb98396e9dc17fbe9b0da2b1bd35d9b05fff30c build -q && readelf -p .comment ./target/debug/helloworld

String dump of section '.comment':
  [     0]  GCC: (Ubuntu 11.3.0-1ubuntu1~22.04.1) 11.3.0
  [    2d]  Linker: LLD 16.0.5

Looks like yes?

rust-timer · 2023-07-05T23:34:04Z

Finished benchmarking commit (4cb98396e9dc17fbe9b0da2b1bd35d9b05fff30c): comparison URL.

Overall result: ✅ improvements - no action needed

Benchmarking this pull request likely means that it is perf-sensitive, so we're automatically marking it as not fit for rolling up. While you can manually mark this PR as fit for rollup, we strongly recommend not doing so since this PR may lead to changes in compiler perf.

@bors rollup=never
@rustbot label: -S-waiting-on-perf -perf-regression

Warning ⚠: The following benchmark(s) failed to build:

rustc

Instruction count

This is a highly reliable metric that was used to determine the overall result at the top of this comment.

	mean	range	count
Regressions ❌ (primary)	-	-	0
Regressions ❌ (secondary)	-	-	0
Improvements ✅ (primary)	-34.1%	[-74.8%, -0.6%]	31
Improvements ✅ (secondary)	-32.7%	[-74.0%, -3.7%]	74
All ❌✅ (primary)	-34.1%	[-74.8%, -0.6%]	31

Max RSS (memory usage)

Results

This is a less reliable metric that may be of interest but was not used to determine the overall result at the top of this comment.

	mean	range	count
Regressions ❌ (primary)	27.8%	[21.9%, 33.7%]	2
Regressions ❌ (secondary)	2.3%	[2.3%, 2.3%]	1
Improvements ✅ (primary)	-	-	0
Improvements ✅ (secondary)	-	-	0
All ❌✅ (primary)	27.8%	[21.9%, 33.7%]	2

Cycles

Results

This is a less reliable metric that may be of interest but was not used to determine the overall result at the top of this comment.

	mean	range	count
Regressions ❌ (primary)	-	-	0
Regressions ❌ (secondary)	-	-	0
Improvements ✅ (primary)	-32.3%	[-65.9%, -1.4%]	28
Improvements ✅ (secondary)	-28.4%	[-64.8%, -3.3%]	73
All ❌✅ (primary)	-32.3%	[-65.9%, -1.4%]	28

Binary size

Results

This is a less reliable metric that may be of interest but was not used to determine the overall result at the top of this comment.

	mean	range	count
Regressions ❌ (primary)	1.6%	[0.6%, 2.4%]	20
Regressions ❌ (secondary)	0.5%	[0.4%, 1.1%]	61
Improvements ✅ (primary)	-	-	0
Improvements ✅ (secondary)	-0.3%	[-0.4%, -0.1%]	2
All ❌✅ (primary)	1.6%	[0.6%, 2.4%]	20

Bootstrap: missing data

lqd · 2023-07-18T17:42:43Z

The data gathering is done so I'll close this for now, but may reopen to rerun crater on these new try artifacts.

The suboptimal error only appears with NLLs due to liveness differences where polonius cannot have as many boring locals.

lqd · 2025-06-16T14:55:47Z

let's see what the currently worst case is

@bors2 try @rust-timer queue

rust-bors · 2025-06-16T14:55:51Z

⌛ Trying commit 55927af with merge e574083…

To cancel the try build, run the command @bors2 try cancel.

[perf] test MCP510 r? `@ghost`

rust-bors · 2025-06-16T17:18:27Z

☀️ Try build successful (CI)
Build commit: e574083 (e574083192d6371fe59e576a2ca0e93d0f9dffd0, parent: d9ca9bd014074e2bac567eaa2b66bfacb2591028)

rust-timer · 2025-06-16T20:42:09Z

Finished benchmarking commit (e574083): comparison URL.

Overall result: ❌ regressions - BENCHMARK(S) FAILED

Benchmarking this pull request means it may be perf-sensitive – we'll automatically label it not fit for rolling up. You can override this, but we strongly advise not to, due to possible changes in compiler perf.

Next Steps: If you can justify the regressions found in this try perf run, please do so in sufficient writing along with @rustbot label: +perf-regression-triaged. If not, please fix the regressions and do another perf run. If its results are neutral or positive, the label will be automatically removed.

@bors rollup=never
@rustbot label: -S-waiting-on-perf +perf-regression

❗ ❗ ❗ ❗ ❗
Warning ⚠️: The following benchmark(s) failed to build:

coercions

❗ ❗ ❗ ❗ ❗

Instruction count

Our most reliable metric. Used to determine the overall result above. However, even this metric can be noisy.

	mean	range	count
Regressions ❌ (primary)	5.9%	[0.1%, 63.8%]	196
Regressions ❌ (secondary)	1625.3%	[0.1%, 61565.9%]	170
Improvements ✅ (primary)	-	-	0
Improvements ✅ (secondary)	-0.8%	[-0.9%, -0.6%]	2
All ❌✅ (primary)	5.9%	[0.1%, 63.8%]	196

Max RSS (memory usage)

Results (primary 21.3%, secondary 38.6%)

A less reliable metric. May be of interest, but not used to determine the overall result above.

	mean	range	count
Regressions ❌ (primary)	21.3%	[0.9%, 65.1%]	26
Regressions ❌ (secondary)	47.4%	[0.9%, 189.9%]	39
Improvements ✅ (primary)	-	-	0
Improvements ✅ (secondary)	-4.2%	[-8.3%, -1.3%]	8
All ❌✅ (primary)	21.3%	[0.9%, 65.1%]	26

Cycles

Results (primary 9.2%, secondary 3091.4%)

A less reliable metric. May be of interest, but not used to determine the overall result above.

	mean	range	count
Regressions ❌ (primary)	9.2%	[0.7%, 63.2%]	98
Regressions ❌ (secondary)	3091.4%	[1.2%, 63942.2%]	91
Improvements ✅ (primary)	-	-	0
Improvements ✅ (secondary)	-	-	0
All ❌✅ (primary)	9.2%	[0.7%, 63.2%]	98

Binary size

This benchmark run did not return any relevant results for this metric.

Bootstrap: 756.489s -> 802.349s (6.06%)
Artifact size: 372.07 MiB -> 371.09 MiB (-0.26%)

lqd · 2025-06-18T09:03:01Z

@bors2 try @rust-timer queue

rust-bors · 2025-06-18T09:03:05Z

⌛ Trying commit 0ef9541 with merge 1065fec…

To cancel the try build, run the command @bors2 try cancel.

[perf] test MCP510 r? `@ghost`

rust-log-analyzer · 2025-06-18T09:33:53Z

The job x86_64-gnu-tools failed! Check out the build log: (web) (plain)

Click to see the possible cause of the failure (guessed by this bot)

rust-bors · 2025-06-18T11:37:26Z

☀️ Try build successful (CI)
Build commit: 1065fec (1065fec1c91b6e2377664ddf17ca4bea80a022f6, parent: 27733d46d79f4eb92e240fbba502c43022665735)

rust-timer · 2025-06-18T17:41:15Z

Finished benchmarking commit (1065fec): comparison URL.

Overall result: ❌ regressions - please read the text below

Benchmarking this pull request means it may be perf-sensitive – we'll automatically label it not fit for rolling up. You can override this, but we strongly advise not to, due to possible changes in compiler perf.

Next Steps: If you can justify the regressions found in this try perf run, please do so in sufficient writing along with @rustbot label: +perf-regression-triaged. If not, please fix the regressions and do another perf run. If its results are neutral or positive, the label will be automatically removed.

@bors rollup=never
@rustbot label: -S-waiting-on-perf +perf-regression

Instruction count

Our most reliable metric. Used to determine the overall result above. However, even this metric can be noisy.

	mean	range	count
Regressions ❌ (primary)	6.0%	[0.1%, 63.0%]	189
Regressions ❌ (secondary)	7.0%	[0.2%, 57.4%]	156
Improvements ✅ (primary)	-	-	0
Improvements ✅ (secondary)	-0.3%	[-0.4%, -0.2%]	2
All ❌✅ (primary)	6.0%	[0.1%, 63.0%]	189

Max RSS (memory usage)

Results (primary 16.8%, secondary 30.7%)

A less reliable metric. May be of interest, but not used to determine the overall result above.

	mean	range	count
Regressions ❌ (primary)	19.8%	[1.0%, 64.2%]	28
Regressions ❌ (secondary)	40.6%	[1.7%, 186.2%]	46
Improvements ✅ (primary)	-4.2%	[-12.2%, -1.0%]	4
Improvements ✅ (secondary)	-4.1%	[-8.6%, -1.1%]	13
All ❌✅ (primary)	16.8%	[-12.2%, 64.2%]	32

Cycles

Results (primary 9.5%, secondary 10.6%)

A less reliable metric. May be of interest, but not used to determine the overall result above.

	mean	range	count
Regressions ❌ (primary)	9.5%	[0.9%, 62.1%]	91
Regressions ❌ (secondary)	10.6%	[1.4%, 54.1%]	87
Improvements ✅ (primary)	-	-	0
Improvements ✅ (secondary)	-	-	0
All ❌✅ (primary)	9.5%	[0.9%, 62.1%]	91

Binary size

This benchmark run did not return any relevant results for this metric.

Bootstrap: 693.071s -> 732.812s (5.73%)
Artifact size: 372.10 MiB -> 371.07 MiB (-0.28%)

rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Jul 5, 2023

This comment has been minimized.

Sign in to view

rustbot added the S-waiting-on-perf Status: Waiting on a perf run to be completed. label Jul 5, 2023

This comment was marked as outdated.

Sign in to view

This comment was marked as duplicate.

Sign in to view

This comment has been minimized.

Sign in to view

rustbot removed the S-waiting-on-perf Status: Waiting on a perf run to be completed. label Jul 5, 2023

lqd closed this Jul 18, 2023

lqd reopened this Sep 12, 2023

lqd force-pushed the test-mcp510 branch from 793c8e6 to a9e047f Compare September 12, 2023 15:38

rustbot added the T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap) label Sep 12, 2023

This comment has been minimized.

Sign in to view

lqd force-pushed the test-mcp510 branch from a9e047f to 68e4f7d Compare September 12, 2023 17:25

lqd force-pushed the test-mcp510 branch from 68e4f7d to fb7cde3 Compare September 20, 2023 17:39

This comment was marked as resolved.

Sign in to view

lqd force-pushed the test-mcp510 branch from fb7cde3 to 3adfc22 Compare September 20, 2023 17:47

This comment was marked as outdated.

Sign in to view

lqd force-pushed the test-mcp510 branch from 3adfc22 to df25e9c Compare September 20, 2023 18:04

This comment was marked as resolved.

Sign in to view

lqd force-pushed the test-mcp510 branch from df25e9c to 03e476e Compare September 20, 2023 19:19

lqd closed this Sep 20, 2023

lqd reopened this Sep 21, 2023

lqd force-pushed the test-mcp510 branch from 03e476e to 5e67656 Compare September 21, 2023 18:31

rustbot added the A-compiletest Area: The compiletest test runner label Jun 16, 2025

lqd added 6 commits June 16, 2025 14:54

tmp: switch polonius compare mode to polonius=next

86dc62a

update test expectations for boring locals + dropckoutlives interactions

fcb22db

The suboptimal error only appears with NLLs due to liveness differences where polonius cannot have as many boring locals.

simplify polonius=next

baf4f6b

cut scope for more complicated cases of flow-sensitivity

6432f89

new impl fixes crash test

dd1fa38

tmp: how trash is this prototype currently?

55927af

lqd force-pushed the test-mcp510 branch from af1b6e3 to 55927af Compare June 16, 2025 14:54

This comment has been minimized.

Sign in to view

rust-bors bot added a commit that referenced this pull request Jun 16, 2025

Auto merge of #113382 - lqd:test-mcp510, r=<try>

e574083

[perf] test MCP510 r? `@ghost`

rustbot added the S-waiting-on-perf Status: Waiting on a perf run to be completed. label Jun 16, 2025

This comment has been minimized.

Sign in to view

rustbot added perf-regression Performance regression. and removed S-waiting-on-perf Status: Waiting on a perf run to be completed. labels Jun 16, 2025

turn expensive assert into debug assertion

0ef9541

This comment has been minimized.

Sign in to view

rust-bors bot added a commit that referenced this pull request Jun 18, 2025

Auto merge of #113382 - lqd:test-mcp510, r=<try>

1065fec

[perf] test MCP510 r? `@ghost`

rustbot added the S-waiting-on-perf Status: Waiting on a perf run to be completed. label Jun 18, 2025

This comment has been minimized.

Sign in to view

rustbot removed the S-waiting-on-perf Status: Waiting on a perf run to be completed. label Jun 18, 2025

[perf] test MCP510 #113382

Are you sure you want to change the base?

[perf] test MCP510 #113382

Conversation

lqd commented Jul 5, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

lqd commented Jul 5, 2023

Uh oh!

This comment has been minimized.

This comment was marked as outdated.

This comment was marked as outdated.

This comment was marked as duplicate.

This comment has been minimized.

lqd commented Jul 5, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

rust-timer commented Jul 5, 2023

Overall result: ✅ improvements - no action needed

Uh oh!

lqd commented Jul 18, 2023

Uh oh!

This comment has been minimized.

This comment was marked as resolved.

This comment was marked as outdated.

This comment was marked as resolved.

lqd commented Jun 16, 2025

Uh oh!

This comment has been minimized.

rust-bors bot commented Jun 16, 2025

Uh oh!

This comment has been minimized.

rust-bors bot commented Jun 16, 2025

Uh oh!

This comment has been minimized.

rust-timer commented Jun 16, 2025

Overall result: ❌ regressions - BENCHMARK(S) FAILED

Uh oh!

lqd commented Jun 18, 2025

Uh oh!

This comment has been minimized.

rust-bors bot commented Jun 18, 2025

Uh oh!

rust-log-analyzer commented Jun 18, 2025

Uh oh!

rust-bors bot commented Jun 18, 2025

Uh oh!

This comment has been minimized.

rust-timer commented Jun 18, 2025

Overall result: ❌ regressions - please read the text below

Uh oh!

Uh oh!

lqd commented Jul 5, 2023 •

edited

Loading

lqd commented Jul 5, 2023 •

edited

Loading